Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a few errors checks #119

Merged
merged 9 commits into from
Apr 22, 2021
Merged

Conversation

AngheloAlf
Copy link
Contributor

Add checks for a few errors. If a check fails, then an exception would be raised.

  • Check if an unknown resource was declared in a XML file.
  • Check if a resource was declared outside of a File node.
  • Check if a File was declared inside a File node.
  • Check if a resource has childs, and throw an exception if that resource shouldn't.
  • Check for repeated names, outnames and offsets.
  • Check for valid variable name for each resource.

This PR is related to issue #18

@NEstelami NEstelami merged commit b0d98ff into NEstelami:master Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants